🌱 refactor: simplify scroll management and add useContainerWidth hook#1393
🌱 refactor: simplify scroll management and add useContainerWidth hook#1393ibolton336 wants to merge 2 commits intokonveyor:mainfrom
Conversation
When the LLM returns no response, log at warn level with actionable messages instead of silly level. Helps diagnose misconfigured model providers. Affected nodes: AnalysisIssueFix, SummarizeHistory, PlanFixes, FixGeneralIssues, FixJavaDependencyIssues Signed-off-by: Ian Bolton <ibolton@redhat.com>
Refactors useScrollManagement to be more reliable: - Cleaner auto-scroll logic with threshold-based detection - Better handling of user scroll intent vs programmatic scrolls - Simplified state management Adds useContainerWidth hook for responsive layout breakpoints in chat and resolution panels. Signed-off-by: Ian Bolton <ibolton@redhat.com>
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Review rate limit: 0/1 reviews remaining, refill in 45 minutes and 48 seconds.Comment |
…yor#1393, konveyor#1394) These changes are now in standalone PRs that should merge first: - konveyor#1393 — scroll management refactor + useContainerWidth - konveyor#1394 — ToolMessage UI refactor Signed-off-by: Ian Bolton <ibolton@redhat.com>
ibolton336
changed the title
…yor#1393, konveyor#1394) These changes are now in standalone PRs that should merge first: - konveyor#1393 — scroll management refactor + useContainerWidth - konveyor#1394 — ToolMessage UI refactor Signed-off-by: Ian Bolton <ibolton@redhat.com>
- Add mcpBridgeServer.ts stub (full impl in PR konveyor#1396) - Adapt checkBatchReviewComplete to use state.mutate() API (konveyor#1386 compat) - Revert removeListener back to removeAllListeners (KaiWorkflowEventEmitter lacks it) - Remove duplicate fileUriToPath import in handleFileResponse - Include useContainerWidth hook (from PR konveyor#1393) - Include updated useScrollManagement (from PR konveyor#1393) - Include ToolMessage with CollapsibleToolGroup/AgentToolGroup exports (from PR konveyor#1394) Signed-off-by: Ian Bolton <ibolton@redhat.com>
1 participant
Summary
Refactors the scroll management hook and adds a responsive container width hook.
Changes
webview-ui/src/hooks/useScrollManagement.ts:webview-ui/src/hooks/useContainerWidth.ts(new):Testing